45aae5fedf670f0fbe49b4a867c3bcf67f03bcab,clc/modules/walrus/src/main/java/edu/ucsb/eucalyptus/cloud/ws/WalrusManager.java,WalrusManager,getBucketAccessControlPolicy,#GetBucketAccessControlPolicyType#,564
Before Change
try {
//TODO: zhill - Modify this to handle invalid accounts and just skip them. This is just response creation, not authorization
if (uId != null) {
addPermission(grants, Accounts.lookupAccountById(uId), grantInfo);
} else {
addPermission(grants, grantInfo);
}
After Change
//Lots of work just to try to get the display Name of the userId.
//TODO: zhill - Operations like this shouldn't have to hit the DB for every user, that can be in the grant record
try {
tmpAccnt = Accounts.lookupAccountById(uId);
grantUser = new CanonicalUserType(tmpAccnt.getAccountNumber(), tmpAccnt.getName());
tmpAccnt = null;
} catch(AuthException e) {
LOG.debug(e,e);